-
Notifications
You must be signed in to change notification settings - Fork 1
[FEATURE] 이미지 업로드 컨트롤러 추가 #345
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
빌드에 실패했습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
빌드에 실패했습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
빌드에 실패했습니다.
|
ayoung-dev
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니당
7zrv
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다!



resolved :
📌 과제 설명
서진 님과 상의해서 이미지 업로드 로직을 정리할 계획입니다.
주석과 PR 포인트를 고려해서 리뷰 부탁드리겠습니다!
👩💻 요구 사항과 구현 내용
이미지 업로드 컨트롤러 추가
✅ PR 포인트 & 궁금한 점
이미지 업로드 기록 방식
userId,fileName,presignedUrl을 로그로 기록하여 추적 가능.PENDING상태로 DB에 저장.UPLOADED상태로 변경.userId포함userId + timestamp + UUID조합을 활용하여 추적 가능.SHA256(userId)활용 가능.배드 케이스 & 해결 방법
개선된 설계 흐름
1️⃣ Presigned URL 발급 시, DB에
PENDING상태로 저장2️⃣ S3 Event Notification을 활용하여
UPLOADED상태 자동 반영3️⃣ 클라이언트가 업로드 완료 후
POST /api/image/confirm호출하여 상태 변경4️⃣ 배치 작업을 통해
PENDING상태가 오래된 데이터를 삭제